Skip to content

Conversation

@swolchok
Copy link
Contributor

@swolchok swolchok commented Feb 11, 2025

Hopefully, this should help prevent mergebot PRs from being unnoticed by generating a GitHub notification to the author.

Test Plan: Use ghstack to push a PR (get something like gh/kirklandsign/17/head).
In .github/scripts/propose_ghstack_orig_pr.py, remove print("The PR (and stack above) is not merged yet, skipping") and return after it.
Then export GITHUB_TOKEN= python .github/scripts/propose_ghstack_orig_pr.py --repo pytorch/executorch --ref refs/heads/gh/kirklandsign/17/head

[ghstack-poisoned]
@swolchok
Copy link
Contributor Author

swolchok commented Feb 11, 2025

Stack from ghstack (oldest at bottom):

@pytorch-bot
Copy link

pytorch-bot bot commented Feb 11, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/8390

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit e7f83c0 with merge base 85231f5 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 11, 2025
swolchok added a commit that referenced this pull request Feb 11, 2025
Hopefully, this should help prevent mergebot PRs from being unnoticed by generating a GitHub notification to the author.

Test Plan: please advise; I don't know how to test this bot

ghstack-source-id: 02112a9
ghstack-comment-id: 2652256227
Pull Request resolved: #8390
@swolchok swolchok merged commit 3b3ff5c into main Feb 12, 2025
46 of 48 checks passed
@swolchok swolchok deleted the gh/swolchok/250/head branch February 12, 2025 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants